Add Manga Notes #428
Add Manga Notes #428
Conversation
|
This comment is no longer relevant. With the new rebase on main as well as the 0e0a9aa (Make spotless) commit (as well as the detekt gradle task) the information in this comment no longer makes any sense. Its contents are hidden here for posterity. HiddenCommenting on the remaining failing
Let me know if there are any important issues that need to be fixed. Footnotes
|
|
I've created a mock-up for this. User interaction would be:
|
|
Why not view and edit notes buttons?(Instead of only the view button) |
|
Hi @CrepeTF . I'm not sure if putting it in a outlined rounded rectangle is a good idea. The padding on the sides removes vertical space that can be used for text and the button is quite distracting as I believe the text of your notes are should be first and foremost. On the other hand, I like how attention grabbing it is and draws your attention to whatever you wrote about the manga. Perhaps there is a middle ground somewhere? It might be weird to have a big colored notes box when there are no notes added yet. |
pending work is to save the notes and make it work with the backup
- moved note to manga screen - added backup support - make notes save - change the way the initial model is loaded
|
Rebased off of main, prevent silly merge commit from messing everything up. :) |
|
Good to see you're still motivated despite me not reviewing. If you can hop on to Mihon discord for design discussion that'll be great. |
see the Discord for more details
Thank you Syer10 for recommending this library - Already built component for elegant rich text editing - Adds a dependency, could possibly remove another (NewUpdateScreen.kt's markdown viewer)
|
Updated demonstration video with recent changes: Preview.mp4 |
Foreword
This is my first time ever working on anything Android related (the time of me downloading Android Studio is about two days ago), so any feedback on how to improve or pointing out any pitfalls is appreciated. This is also my first time contributing to a medium-sized open source project.
Manga Notes
What
This feature adds built-in per-manga
markdown1 notes to help remember the details of manga inside and outside your library. It can be accessed through the top overflow menu or a button just below the tags through a button.Why
Picture this: you have 50 or so forgettable isekai2 manga in your library. Because they are all in fact the same story with slight modifications, you lose track of the individual differences in each manga.
With the notes feature, you can add small (or large) reminders about the plot to each (easily) forgettable manga to make the reading experience of new chapters not completely horrible.
Intentions
I will commit to helping maintain this feature (when necessary) and improve this feature as to not place undue burden upon existing project maintainers.
Known issues
No collapsing/expanding notes on manga screenAddressed: Recent design revisions have the manga notes in the collapsible description section
No limit on note size (in theory could work correctly with notes2^31-1characters long, but not tested)Addressed: Imposed a 10k length limit with a visible warning when close to limit (tested)
TextFields and its auto-suggestion selects 1 space character extra behind the current word (not related to this feature itself)Demonstration
Revised.mp4
Footnotes
Markdown was chosen over something like rich text for being just enough for formatting if ever needed without requiring any additional bundle overhead without interfering with normal text.EDIT: The current revision uses rich text (HTML) for compatibility with the WYSIWYG editor ↩
Meaning: "Another world"↩